Skip to content

fix: Trailing comma causing JSON_PARSING_ERROR#4815

Merged
amarzavery merged 1 commit intoAzure:masterfrom
nschonni:patch-1
Dec 8, 2018
Merged

fix: Trailing comma causing JSON_PARSING_ERROR#4815
amarzavery merged 1 commit intoAzure:masterfrom
nschonni:patch-1

Conversation

@nschonni
Copy link
Copy Markdown
Contributor

@nschonni nschonni commented Dec 6, 2018

Not sure why the build in #4344 didn't fail when it was added, but the semantic PR_Only=false can't parse the file with the extra comma

@openapi-portal-comment
Copy link
Copy Markdown

If you're a MSFT employee, click this link
to view this PR's validation status on our new OpenAPI Hub spec management tool.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-python

Unable to detect any generation context from this PR.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-go

Unable to detect any generation context from this PR.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-java

Unable to detect any generation context from this PR.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-js

Unable to detect any generation context from this PR.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-ruby

Unable to detect any generation context from this PR.

@AutorestCI
Copy link
Copy Markdown

AutorestCI commented Dec 6, 2018

Automation for azure-sdk-for-node

Unable to detect any generation context from this PR.

@azuresdkci
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@amarzavery amarzavery merged commit 1d18cfd into Azure:master Dec 8, 2018
@nschonni nschonni deleted the patch-1 branch December 8, 2018 04:06
TalluriAnusha pushed a commit to AsrOneSdk/azure-rest-api-specs that referenced this pull request Feb 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants